Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Range slider component #78

Merged
merged 6 commits into from
Aug 16, 2018
Merged

Range slider component #78

merged 6 commits into from
Aug 16, 2018

Conversation

y0hami
Copy link
Member

@y0hami y0hami commented Jul 28, 2018

Added the range slider component made by @tyleryasaka.

Range repo issue: tyleryasaka/semantic-ui-range#30

Closed Issues

Semantic-Org/Semantic-UI#1336

@y0hami y0hami added type/feat Any feature requests or improvements lang/css Anything involving CSS lang/javascript Anything involving JavaScript tag/new-component New UI components state/awaiting-reviews Pull requests which are waiting for reviews state/awaiting-docs Pull requests which need doc changes/additions labels Jul 28, 2018
@y0hami y0hami requested review from ColinFrick and prudho July 28, 2018 01:38
@@ -0,0 +1,273 @@
/*!
* # Range slider for Semantic UI.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you set the standard header ? Here it is:

/*!
 * # Semantic UI - Range slider
 * http://github.com/semantic-org/semantic-ui/
 *
 *
 * Released under the MIT license
 * http://opensource.org/licenses/MIT
 *
 */

},

};

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a blocking stuff, but we should add later common modules methods (debug, performance, display, verbose...).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good shout, didn't think of that

prudho
prudho previously approved these changes Aug 13, 2018
ColinFrick
ColinFrick previously approved these changes Aug 14, 2018
@y0hami y0hami added state/awaiting-changes Anything which is awaiting changes and removed state/awaiting-reviews Pull requests which are waiting for reviews labels Aug 14, 2018
@y0hami y0hami dismissed stale reviews from ColinFrick and prudho via 704b1e1 August 16, 2018 13:42
@y0hami y0hami requested review from ColinFrick and prudho August 16, 2018 13:43
@y0hami y0hami removed the state/awaiting-changes Anything which is awaiting changes label Aug 16, 2018
@y0hami y0hami merged commit 2e1b765 into beta Aug 16, 2018
@y0hami y0hami deleted the range-slider branch August 16, 2018 13:45
@ColinFrick ColinFrick mentioned this pull request Aug 19, 2018
@y0hami y0hami added state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo and removed state/awaiting-docs Pull requests which need doc changes/additions labels Sep 2, 2018
@y0hami y0hami mentioned this pull request Sep 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/css Anything involving CSS lang/javascript Anything involving JavaScript state/has-docs A issue/PR which requires documentation changes and has the corresponding PR open in the docs repo tag/new-component New UI components type/feat Any feature requests or improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants